docs(5956): add documentation to sentry usage#38644
Merged
HowardBraham merged 9 commits intomainfrom Jan 7, 2026
Merged
Conversation
Contributor
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
DDDDDanica
force-pushed
the
chore/5956-sentry-doc
branch
from
80c4ceb to
132655d
Compare
DDDDDanica
force-pushed
the
chore/5956-sentry-doc
branch
from
132655d to
4563265
Compare
Collaborator
Builds ready [4563265]
UI Startup Metrics (1252 ± 82 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Comment on lines
+78
to
+86
| | Environment | When Used | Build Command | Branch/Context | | ||
| | ------------------- | ---------------------------- | ----------------- | -------------------- | | ||
| | `production` | Production releases | `yarn build prod` | `stable` branch | | ||
| | `staging` | Main branch builds | `yarn build dist` | `main` branch | | ||
| | `development` | Local development | `yarn start` | Local dev server | | ||
| | `testing` | E2E test builds | `yarn build:test` | Any | | ||
| | `pull-request` | PR builds | `yarn build dist` | `pull_request` event | | ||
| | `release-candidate` | Release branches | `yarn build dist` | `release/*` branches | | ||
| | `other` | Fallback (should not happen) | - | Uncategorized | |
Contributor
There was a problem hiding this comment.
Should we give the webpack commands too?
DDDDDanica
temporarily deployed
to
pr-comment
GitHub Actions
Inactive
DDDDDanica
temporarily deployed
to
pr-comment
GitHub Actions
Inactive
Contributor
Builds ready [85f37a4]
UI Startup Metrics (1337 ± 110 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚀 Bundle size reduced!]
|
Gudahtt
reviewed
Dec 19, 2025
| ; The below is the `test-metamask` project. | ||
| ; SENTRY CONFIGURATION | ||
| ; SENTRY_DSN - For production builds only (do NOT use in development) | ||
| ; SENTRY_DSN_DEV - For local dev, CI/CD, and testing (sends to 'test-metamask' project) |
Member
There was a problem hiding this comment.
Is this used by CI? I recall @HowardBraham was using this for performance testing at one point, I don't know if that's still the case.
Member
There was a problem hiding this comment.
We may want to reconsider the decision to make the testing DSN publicly available given the amount of traffic it gets. We may also want to consider creating a separate project for CI than for testing/development.
When we first created test-metamask, it was easy to use because typically the person testing was the only one using the project at any given time, so errors were easy to detect.
Today given the volume and rate limiting, it's far more difficult to make use of.
If we created a new project just for one-off testing (for development or QA), and kept it secret, and avoided using it in CI, then we'd be back to the easy-to use experience we once had.
Contributor
There was a problem hiding this comment.
Oh yes, we have a new just-for-CI one, it's called SENTRY_DSN_PERFORMANCE
Contributor
Author
There was a problem hiding this comment.
Included SENTRY_DSN_PERFORMANCE in 12c0844
DDDDDanica
had a problem deploying
to
pr-comment
GitHub Actions
Failure
DDDDDanica
had a problem deploying
to
pr-comment
GitHub Actions
Failure
Contributor
Builds ready [594e3a8]
UI Startup Metrics (1296 ± 114 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Contributor
Builds ready [de385bb]
UI Startup Metrics (1318 ± 112 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
HowardBraham
changed the title
Contributor
Builds ready [03d48ba]
UI Startup Metrics (1269 ± 101 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Gudahtt
reviewed
Jan 7, 2026
Gudahtt
reviewed
Jan 7, 2026
Gudahtt
reviewed
Jan 7, 2026
Gudahtt
reviewed
Jan 7, 2026
Contributor
Builds ready [38b94cd]
UI Startup Metrics (1311 ± 99 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
HowardBraham
approved these changes
Jan 7, 2026
metamaskbot
added
the
release-13.14.0
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
The
test-metamaskSentry project receives significant traffic (284,823 events in 14 days), but there's no clear documentation on sentry setup and usage. This PR add some comprehensive documentation that developers can reference.Changelog
CHANGELOG entry: null
Related issues
Fixes: https://github.com/MetaMask/MetaMask-planning/issues/5956
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Enhances Sentry documentation and configuration for developers.
development/README.mdwith Sentry overview, project/DSN mapping, environment matrix, environment and DSN selection logic (development/build/utils.js:getEnvironment,app/scripts/lib/setupSentry.js:getSentryTarget), common query examples, and local dev setup/testing steps.metamaskrc.distwith a dedicated Sentry section includingSENTRY_DSN,SENTRY_DSN_DEV(example), andSENTRY_DSN_PERFORMANCE, plus guidance to avoid using production DSN in dev/CIWritten by Cursor Bugbot for commit 38b94cd. This will update automatically on new commits. Configure here.